Skip to content

Add Changelog project URL - #1630

Open
twelfthlabor wants to merge 3 commits into
dbcli:mainfrom
twelfthlabor:fix/changelog-url-1627
Open

Add Changelog project URL#1630
twelfthlabor wants to merge 3 commits into
dbcli:mainfrom
twelfthlabor:fix/changelog-url-1627

Conversation

@twelfthlabor

@twelfthlabor twelfthlabor commented Sep 7, 2026

Copy link
Copy Markdown

Description

PyPI shows only a Homepage link for pgcli, so the changelog is hard to find. This adds a Changelog project URL pointing at changelog.rst on main, following mycli.

Related to #1627 (PyPI portion only; the pgcli.com docs link lives in a separate repo).

Verification

Built wheels from baseline and this patch. Baseline METADATA lists Homepage only; patched METADATA adds Project-URL: Changelog. ruff check and ruff format pass. No Python changed, so no new tests.

Checklist

  • Verified as described above
  • Please squash merge

Comment thread pyproject.toml Outdated
Co-authored-by: Damien Baty <damien@damienbaty.com>
Comment thread pyproject.toml Outdated
Homepage = "https://pgcli.com"
Changelog = "https://github.com/dbcli/pgcli/blob/main/changelog.rst"
Repository = "https://github.com/dbcli/pgcli/"
Issues = "https://github.com/dbcli/pgcli/issues"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did you test your change?

$ git clone git@github.com:twelfthlabor/pgcli.git
[...]
$ cd pgcli
$ git checkout fix/changelog-url-1627 
$ uv build
Building source distribution...
error: Failed to build `/home/damien/dev/perso/a/pgcli`
  Caused by: Failed to parse: `pyproject.toml`
  Caused by: TOML parse error at line 1, column 1
      |
    1 | [project]
      | ^^^^^^^^^
    `pyproject.toml` is using the `[project]` table, but the required `project.version` field is neither set nor present in the `project.dynamic` list

The [project.urls] block must not be in the middle of other statements. It should appear below the definition of dynamic.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, I accepted the suggestion without rerunning the build. Fixed the table placement and verified uv build passes now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants